Skip to content

Conversation

@rgraber
Copy link
Contributor

@rgraber rgraber commented Nov 5, 2025

🗒️ Checklist

  1. run linter locally
  2. update developer docs (API, README, inline, etc.), if any
  3. for user-facing doc changes create a Zulip thread at #Support Docs Updates, if any
  4. draft PR with a title <type>(<scope>)<!>: <title> DEV-1234
  5. assign yourself, tag PR: at least Front end and/or Back end or workflow
  6. fill in the template below and delete template comments
  7. review thyself: read the diff and repro the preview as written
  8. open PR & confirm that CI passes & request reviewers, if needed
  9. delete this section before merging

💭 Notes

Port selected tests over from test_submission_extras_api_post.py in the old app. Which tests to port were chosen based on whether or not they applied in the new format and if they already had a reasonable equivalent, which many did.
Also cleans up some old tests that were failing or had misleading comments.

@rgraber rgraber changed the base branch from main to refactor-subsequences-2025 November 5, 2025 20:31
@rgraber rgraber self-assigned this Nov 5, 2025
@rgraber rgraber marked this pull request as ready for review November 5, 2025 20:41
@rgraber rgraber requested a review from Guitlle as a code owner November 5, 2025 20:41
@rgraber rgraber removed the request for review from Guitlle November 6, 2025 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants